Skip to content

Comments

Refactor shapes (DUPE FOR CI)#19

Merged
inducer merged 51 commits intomasterfrom
refactor-shapes
Dec 11, 2020
Merged

Refactor shapes (DUPE FOR CI)#19
inducer merged 51 commits intomasterfrom
refactor-shapes

Conversation

@inducer
Copy link
Owner

@inducer inducer commented Nov 28, 2020 •
edited
Loading

Mirror for CI of alexfikl#1

alexfikl and others added 30 commits November 3, 2020 17:09
@alexfikl
make face mass matrices work on tensor products
11bd11f
@alexfikl
fix 0d case for tensor products
cfd8fbf
@alexfikl
remove face index stuff
ffcb91b
@alexfikl
simplify cube unit vertex construction
47dec4d
@alexfikl
simplify face maps a bit
56d7f13
@alexfikl
simplify test setup
ddb7e6d
@alexfikl
update docs
e9b52eb
@alexfikl
introduce a shapes module
704e686
@alexfikl
use shapes in estimate_lebesgue_constant
16180da
@alexfikl
add docs and some py3.6 fixes
3df2de1
@alexfikl
move functions out of shapes module
7acf2b0
@alexfikl
update docs and some fixes
b014fbe
@alexfikl
add basis_with_mode_ids
fe40713
@alexfikl
update docs
3b3bada
@alexfikl
bump version
b90c25c
@alexfikl
estimate_lebesgue_constant: raise if dimensions do not match
0a89725
@inducer
Merge branch 'master' into tensor-face-mass-matrix
773bb2b
@inducer
Move shape-basd functions to bottom in modes/nodes
00488bd
@inducer
Move shapes docstring into shapes module
f227cac
@inducer
Shapes: Add foldmethod comment
f8209ce
@inducer
Remove spurious blank line in modes docstring
7238c6c
@inducer
Use _cse in PKDO gradients
918342a
@inducer
Merge branch 'master' into tensor-face-mass-matrix
acf4b34
@inducer
Deprecate current dimension-independent basis getters
59198d8
@inducer
resampling_matrix: Impove count mismatch error message
cf11d52
@inducer
Refactor towards improved shape-based interface
cd750b8
@inducer
Refactor face information retrieval and face mass matrix computation
560a027
@inducer
Avoid log(0) warnings in test_basis_grad
598145f
@inducer
Reverse vertex/node order for hypercubes
73e8d56
@inducer
Export face query functionality in root namespace
f6c5755
@inducer inducer mentioned this pull request Nov 28, 2020
Merged
2 tasks
inducer and others added 21 commits November 27, 2020 18:17
@inducer
Add redundant @singledispatch.register arguments for Py3.6
e7b8269
@inducer
Fix doc references
c15c453
@inducer
Fix quadrature_for_shape docstring
a15fcfa
@inducer
Eliminate references to (removed) get_node_tuples
8ffc3d4
@inducer
Fix zerod_basis
9061bcc
@inducer
Remove spurious import from modepy.nodes
bceb113
@inducer
Remove an extraneous import
299b32f
@inducer
Make new section for submeshes
c75d799
@inducer @alexfikl
Doc suggestions for refactor-shapes from review by @alexfikl
92e3e83 
Co-authored-by: Alex Fikl <alexfikl@gmail.com>
@inducer @alexfikl
Cast volume_vertex_indices to tuple in _SimplexFace
5e3550a 
Co-authored-by: Alex Fikl <alexfikl@gmail.com>
@inducer
Remove whitespace: placate flake8
22ae991
@inducer
Generalize TensorProductBasis for nD inhomogeneity
7f4e1a1
@inducer
Make TensorProductBasis public in root module
347027b
@inducer
Do not claim that Face subclasses Shape
c872de9
@inducer
Fix TensorProductBasis facepalm
676d38a
@inducer
Introduce function spaces as first-class objects
9a282c0
@inducer
Move submesh functionality into shapes
6bf5735
@inducer
Quadrature, notes: require space *and* shape arguments
62524e2
@inducer
Drop a special case in node_tuples_for_space:Simplex
7f5ec1e
@inducer
Bring back _for_space names for quadrature, nodes
b1f5de4
@inducer
Deprecate tools.unit_vertices, fix biunit_vertices_for_shape for simplex
4917e62
@inducer inducer merged commit 982ae2b into master Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@inducer @alexfikl